Skip to content

Comments

[tado] Add tadoApiUrl configuration parameter#19596

Merged
lsiepel merged 4 commits intoopenhab:mainfrom
ErikDB87:tado_api_proxy_bis
Oct 31, 2025
Merged

[tado] Add tadoApiUrl configuration parameter#19596
lsiepel merged 4 commits intoopenhab:mainfrom
ErikDB87:tado_api_proxy_bis

Conversation

@ErikDB87
Copy link
Contributor

@ErikDB87 ErikDB87 commented Oct 31, 2025

Description

There's a possible by-pass for tado's API limit, as discussed here: https://community.openhab.org/t/tado-api-limits/166127/32. The by-pass is a docker container running chromium, which (hopefully) generates the same token as we humans would when logging in to tado's web app. More about that docker container here: https://github.com/s1adem4n/tado-api-proxy.

For it to work, API calls must be done to another URL than https://my.tado.com. This PR creates the possibility to enter a custom URL.

For full disclosure: see #19588 for a previous version of this PR.

Signed-off-by: Erik De Boeck <deboeck.erik@gmail.com>
@ErikDB87 ErikDB87 requested a review from andrewfg as a code owner October 31, 2025 14:48
@lsiepel lsiepel changed the title [tado] Adds proxy URL option - 2nd attempt [tado] Add tadoApiUrl configuration parameter Oct 31, 2025
@lsiepel lsiepel added the enhancement An enhancement or new feature for an existing add-on label Oct 31, 2025
Copy link
Contributor

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These two comments where left from the previous PR

Signed-off-by: Erik De Boeck <deboeck.erik@gmail.com>
Signed-off-by: Erik De Boeck <deboeck.erik@gmail.com>
Signed-off-by: Erik De Boeck <deboeck.erik@gmail.com>
@ErikDB87 ErikDB87 requested a review from lsiepel October 31, 2025 15:28
@lsiepel lsiepel merged commit 7eae60c into openhab:main Oct 31, 2025
2 checks passed
@lsiepel lsiepel added this to the 5.1 milestone Oct 31, 2025
@ErikDB87 ErikDB87 deleted the tado_api_proxy_bis branch October 31, 2025 16:08
@andrewfg
Copy link
Contributor

@ErikDB87 I see that this PR was merged. But could you please kindly edit the text in the first post of this thread to remove the story about your struggles (which you eventually overcame) and instead simply explain the changes that you actually made. (It helps to keep the record straight for posterity).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement An enhancement or new feature for an existing add-on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants